style: Remove GtkStylePropertyContext again
authorBenjamin Otte <otte@redhat.com>
Mon, 2 Jan 2012 18:22:30 +0000 (19:22 +0100)
committerBenjamin Otte <otte@redhat.com>
Mon, 9 Jan 2012 17:37:56 +0000 (18:37 +0100)
We need to solve this differently. I have no idea yet how, but I'll
invent something later.

This only affects win32 theming and that's broken anyway.

gtk/gtkstylecontext.c
gtk/gtkstylecontextprivate.h
gtk/gtkstyleproperties.c
gtk/gtkstylepropertiesprivate.h
gtk/gtkthemingbackground.c
gtk/gtkthemingbackgroundprivate.h
gtk/gtkthemingengine.c
gtk/gtkthemingengineprivate.h

index a688c1374b8d5c8d8fa92a3543ab12db60285392..6588207d92cfdbe1896860ce270ace042e65022a 100644 (file)
@@ -1440,24 +1440,6 @@ gtk_style_context_get_property (GtkStyleContext *context,
   gtk_style_properties_get_property (data->store, property, 0, value);
 }
 
-void
-_gtk_style_context_get_valist (GtkStyleContext *context,
-                              GtkStateFlags    state,
-                              GtkStylePropertyContext *property_context,
-                              va_list          args)
-{
-  GtkStyleContextPrivate *priv;
-  StyleData *data;
-
-  g_return_if_fail (GTK_IS_STYLE_CONTEXT (context));
-
-  priv = context->priv;
-  g_return_if_fail (priv->widget_path != NULL);
-
-  data = style_data_lookup (context, state);
-  _gtk_style_properties_get_valist (data->store, 0, property_context, args);
-}
-
 /**
  * gtk_style_context_get_valist:
  * @context: a #GtkStyleContext
index 69d619330a20341f30a7f5af620cf175aeff5e03..981eda76e2b23de25de33f87bd1baebba4a5b43c 100644 (file)
 
 G_BEGIN_DECLS
 
-typedef struct _GtkStylePropertyContext GtkStylePropertyContext;
-struct _GtkStylePropertyContext
-{
-  int width;
-  int height;
-};
-
-void           _gtk_style_context_get_valist (GtkStyleContext *context,
-                                             GtkStateFlags    state,
-                                             GtkStylePropertyContext *property_context,
-                                             va_list          args);
 const GValue * _gtk_style_context_peek_style_property        (GtkStyleContext *context,
                                                               GType            widget_type,
                                                               GtkStateFlags    state,
index c7e36aa024ddf5110faf1227669a8e6e2c8c2433..46d537412a3bce2ed3a8f8a715b0de8a08383cef 100644 (file)
@@ -628,12 +628,25 @@ _gtk_style_properties_peek_property (GtkStyleProperties  *props,
   return property_data_match_state (prop, state);
 }
 
+/**
+ * gtk_style_properties_get_property:
+ * @props: a #GtkStyleProperties
+ * @property: style property name
+ * @state: state to retrieve the property value for
+ * @value: (out) (transfer full):  return location for the style property value.
+ *
+ * Gets a style property from @props for the given state. When done with @value,
+ * g_value_unset() needs to be called to free any allocated memory.
+ *
+ * Returns: %TRUE if the property exists in @props, %FALSE otherwise
+ *
+ * Since: 3.0
+ **/
 gboolean
-_gtk_style_properties_get_property (GtkStyleProperties *props,
-                                   const gchar        *property,
-                                   GtkStateFlags       state,
-                                   GtkStylePropertyContext *context,
-                                   GValue             *value)
+gtk_style_properties_get_property (GtkStyleProperties *props,
+                                   const gchar        *property,
+                                   GtkStateFlags       state,
+                                   GValue             *value)
 {
   GtkStyleProperty *node;
 
@@ -658,41 +671,19 @@ _gtk_style_properties_get_property (GtkStyleProperties *props,
 }
 
 /**
- * gtk_style_properties_get_property:
+ * gtk_style_properties_get_valist:
  * @props: a #GtkStyleProperties
- * @property: style property name
- * @state: state to retrieve the property value for
- * @value: (out) (transfer full):  return location for the style property value.
- *
- * Gets a style property from @props for the given state. When done with @value,
- * g_value_unset() needs to be called to free any allocated memory.
+ * @state: state to retrieve the property values for
+ * @args: va_list of property name/return location pairs, followed by %NULL
  *
- * Returns: %TRUE if the property exists in @props, %FALSE otherwise
+ * Retrieves several style property values from @props for a given state.
  *
  * Since: 3.0
  **/
-gboolean
-gtk_style_properties_get_property (GtkStyleProperties *props,
-                                   const gchar        *property,
-                                   GtkStateFlags       state,
-                                   GValue             *value)
-{
-  GtkStylePropertyContext context = { 100, 100};
-
-  g_return_val_if_fail (GTK_IS_STYLE_PROPERTIES (props), FALSE);
-  g_return_val_if_fail (property != NULL, FALSE);
-  g_return_val_if_fail (value != NULL, FALSE);
-
-  return _gtk_style_properties_get_property (props,
-                                            property,
-                                            state, &context, value);
-}
-
 void
-_gtk_style_properties_get_valist (GtkStyleProperties *props,
-                                 GtkStateFlags       state,
-                                 GtkStylePropertyContext *context,
-                                 va_list             args)
+gtk_style_properties_get_valist (GtkStyleProperties *props,
+                                 GtkStateFlags       state,
+                                 va_list             args)
 {
   const gchar *property_name;
 
@@ -705,11 +696,10 @@ _gtk_style_properties_get_valist (GtkStyleProperties *props,
       gchar *error = NULL;
       GValue value = G_VALUE_INIT;
 
-      if (!_gtk_style_properties_get_property (props,
-                                              property_name,
-                                              state,
-                                              context,
-                                              &value))
+      if (!gtk_style_properties_get_property (props,
+                                             property_name,
+                                             state,
+                                             &value))
        break;
 
       G_VALUE_LCOPY (&value, args, 0, &error);
@@ -726,41 +716,6 @@ _gtk_style_properties_get_valist (GtkStyleProperties *props,
     }
 }
 
-/**
- * gtk_style_properties_get_valist:
- * @props: a #GtkStyleProperties
- * @state: state to retrieve the property values for
- * @args: va_list of property name/return location pairs, followed by %NULL
- *
- * Retrieves several style property values from @props for a given state.
- *
- * Since: 3.0
- **/
-void
-gtk_style_properties_get_valist (GtkStyleProperties *props,
-                                 GtkStateFlags       state,
-                                 va_list             args)
-{
-  GtkStylePropertyContext context = { 100, 100};
-  
-  return _gtk_style_properties_get_valist (props, state, &context, args);
-}
-
-void
-_gtk_style_properties_get (GtkStyleProperties *props,
-                          GtkStateFlags       state,
-                          GtkStylePropertyContext *context,
-                          ...)
-{
-  va_list args;
-
-  g_return_if_fail (GTK_IS_STYLE_PROPERTIES (props));
-
-  va_start (args, context);
-  _gtk_style_properties_get_valist (props, state, context, args);
-  va_end (args);
-}
-
 /**
  * gtk_style_properties_get:
  * @props: a #GtkStyleProperties
index cda88101eddc8227004b2b7dc7517aaff944a612..526559cdcc7f3aae86ececdf13f47566deaad9fa 100644 (file)
 
 G_BEGIN_DECLS
 
-void           _gtk_style_properties_get                      (GtkStyleProperties      *props,
-                                                              GtkStateFlags            state,
-                                                              GtkStylePropertyContext *context,
-                                                              ...);
 void           _gtk_style_properties_set_color_lookup_func    (GtkStyleProperties      *props,
                                                                GtkSymbolicColorLookupFunc func,
                                                                gpointer                 data);
-void           _gtk_style_properties_get_valist               (GtkStyleProperties      *props,
-                                                              GtkStateFlags            state,
-                                                              GtkStylePropertyContext *context,
-                                                              va_list                  args);
 
 const GValue * _gtk_style_properties_peek_property            (GtkStyleProperties      *props,
                                                                GtkCssStyleProperty     *property,
index dda56e418f247f06e32246070f30da3f2195552c..9e21110a6b7e74a3b50a481e4ca7dba560fd88ad 100644 (file)
@@ -64,9 +64,9 @@ _gtk_theming_background_apply_running_transformation (GtkThemingBackground *bg,
     other_flags = bg->flags | GTK_STATE_FLAG_PRELIGHT;
 
   gtk_theming_engine_get_background_color (bg->engine, other_flags, &other_bg);
-  _gtk_theming_engine_get (bg->engine, other_flags, &bg->prop_context,
-                           "background-image", &other_pattern,
-                           NULL);
+  gtk_theming_engine_get (bg->engine, other_flags,
+                          "background-image", &other_pattern,
+                          NULL);
 
   if (bg->pattern && other_pattern)
     {
@@ -258,8 +258,6 @@ _gtk_theming_background_apply_origin (GtkThemingBackground *bg)
   }
 
   bg->image_rect = image_rect;
-  bg->prop_context.width = image_rect.width;
-  bg->prop_context.height = image_rect.height;
 }
 
 static void
@@ -389,9 +387,9 @@ _gtk_theming_background_init_engine (GtkThemingBackground *bg)
   _gtk_theming_background_apply_clip (bg);
   _gtk_theming_background_apply_origin (bg);
 
-  _gtk_theming_engine_get (bg->engine, bg->flags, &bg->prop_context, 
-                           "background-image", &bg->pattern,
-                           NULL);
+  gtk_theming_engine_get (bg->engine, bg->flags,
+                          "background-image", &bg->pattern,
+                          NULL);
 }
 
 void
index 09353acd8138da28e2dc7a41472ea951049933e9..540613d79702d694acef789be4a67708e50b7d50 100644 (file)
@@ -50,8 +50,6 @@ struct _GtkThemingBackground {
   GtkBorder border;
   GtkBorder padding;
   GdkRGBA bg_color;
-
-  GtkStylePropertyContext prop_context;
 };
 
 void _gtk_theming_background_init (GtkThemingBackground *bg,
index d0b29dfb95ce84ab8a5f7f49d3b8dcfe9b54d47f..e82453a4bdf4474a3d8b7645e682541d46c6c63b 100644 (file)
@@ -392,25 +392,6 @@ gtk_theming_engine_get_valist (GtkThemingEngine *engine,
   gtk_style_context_get_valist (priv->context, state, args);
 }
 
-void
-_gtk_theming_engine_get (GtkThemingEngine *engine,
-                        GtkStateFlags     state,
-                        GtkStylePropertyContext *property_context,
-                        ...)
-{
-  GtkThemingEnginePrivate *priv;
-  va_list args;
-
-  g_return_if_fail (GTK_IS_THEMING_ENGINE (engine));
-
-  priv = engine->priv;
-
-  va_start (args, property_context);
-  _gtk_style_context_get_valist (priv->context, state, property_context, args);
-  va_end (args);
-}
-
-
 /**
  * gtk_theming_engine_get:
  * @engine: a #GtkThemingEngine
@@ -1556,19 +1537,15 @@ gtk_theming_engine_render_frame (GtkThemingEngine *engine,
   GtkJunctionSides junction;
   GtkBorderImage *border_image;
   GtkBorder border;
-  GtkStylePropertyContext context;
 
   flags = gtk_theming_engine_get_state (engine);
   junction = gtk_theming_engine_get_junction_sides (engine);
   gtk_theming_engine_get_border (engine, flags, &border);
 
-  context.width = width;
-  context.height = height;
-
-  _gtk_theming_engine_get (engine, flags, &context,
-                          "border-image", &border_image,
-                          "border-style", &border_style,
-                          NULL);
+  gtk_theming_engine_get (engine, flags,
+                         "border-image", &border_image,
+                         "border-style", &border_style,
+                         NULL);
 
   if (border_image != NULL)
     {
@@ -1925,7 +1902,6 @@ gtk_theming_engine_render_frame_gap (GtkThemingEngine *engine,
   GtkCssBorderCornerRadius *top_left_radius, *top_right_radius;
   GtkCssBorderCornerRadius *bottom_left_radius, *bottom_right_radius;
   gdouble x0, y0, x1, y1, xc, yc, wc, hc;
-  GtkStylePropertyContext context;
   GtkBorderImage *border_image;
   GtkBorder border;
 
@@ -1933,17 +1909,14 @@ gtk_theming_engine_render_frame_gap (GtkThemingEngine *engine,
   state = gtk_theming_engine_get_state (engine);
   junction = gtk_theming_engine_get_junction_sides (engine);
 
-  context.width = width;
-  context.height = height;
-
   gtk_theming_engine_get_border (engine, state, &border);
-  _gtk_theming_engine_get (engine, state, &context,
-                          "border-image", &border_image,
-                          "border-top-left-radius", &top_left_radius,
-                          "border-top-right-radius", &top_right_radius,
-                          "border-bottom-right-radius", &bottom_right_radius,
-                          "border-bottom-left-radius", &bottom_left_radius,
-                          NULL);
+  gtk_theming_engine_get (engine, state,
+                         "border-image", &border_image,
+                         "border-top-left-radius", &top_left_radius,
+                         "border-top-right-radius", &top_right_radius,
+                         "border-bottom-right-radius", &bottom_right_radius,
+                         "border-bottom-left-radius", &bottom_left_radius,
+                         NULL);
 
   border_width = MIN (MIN (border.top, border.bottom),
                       MIN (border.left, border.right));
index 5191fa05b83245a58406bbdf8f10244cc7eced5f..caf189ec6b3d4c3172a02635d2603b34068ebd0a 100644 (file)
 #define __GTK_THEMING_ENGINE_PRIVATE_H__
 
 #include <gdk/gdk.h>
-#include "gtkstylecontextprivate.h"
-
-void _gtk_theming_engine_get          (GtkThemingEngine *engine,
-                                      GtkStateFlags     state,
-                                      GtkStylePropertyContext *property_context,
-                                      ...) G_GNUC_NULL_TERMINATED;
-
 
 void _gtk_theming_engine_paint_spinner (cairo_t *cr,
                                         gdouble  radius,